Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Editor group not set when reviewer #305

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

f-necas
Copy link
Collaborator

@f-necas f-necas commented Nov 13, 2024

This PR :

  • Adds an Editor entry to usergroups table when Reviewer privilege is set. It avoids not being able to create metadata while being Reviewer.
  • Change logs priority for sync in order to display only changes and useful informations
    image
  • removes GN PR template

geOrchestra/geonetwork checklist

  • PR only involves cherry-picked commits from upstream.
  • PR contains custom code which will soon be available in an upstream release and can be overriden => mention core-geonetwork version if possible.
  • PR contains custom geOrchestra code, which need to be verified during future migrations.

@pmauduit
Copy link
Member

I am surprised this change is needed, because I thought that the Profiles were hierarchical (e.g. Reviewer will give you reviewer rights + all the rights from the lower profiles)

@pmauduit pmauduit merged commit 195790a into georchestra-gn4.4.x Nov 19, 2024
2 of 3 checks passed
@pmauduit pmauduit deleted the fix-editors-gn-sync branch November 19, 2024 17:07
github-actions bot added a commit that referenced this pull request Nov 19, 2024
…-23.0.x/pr-305

[georchestra-gn4.2.x-23.0.x] Merge pull request #305 from georchestra/fix-editors-gn-sync
Copy link

💚 All backports created successfully

Status Branch Result
georchestra-gn4.2.x-23.0.x
georchestra-gn4.2.x-24.0

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

github-actions bot added a commit that referenced this pull request Nov 19, 2024
…-24.0/pr-305

[georchestra-gn4.2.x-24.0] Merge pull request #305 from georchestra/fix-editors-gn-sync
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants